New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Upgrade stripe Python package to the latest version #4904
Conversation
@@ -222,7 +222,7 @@ def test_update_subscription_plan(self): | |||
mock.ANY, | |||
mock.ANY), | |||
mock.call('post', | |||
'/v1/customers/cus_12345/subscriptions/sub_12345', | |||
'/v1/subscriptions/sub_12345', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -69,7 +69,7 @@ def get_subscription(self): | |||
customer = self.get_customer() | |||
|
|||
# TODO get the first subscription more intelligently | |||
subscriptions = customer.subscriptions.all(limit=5) | |||
subscriptions = customer.subscriptions.list(limit=5) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Codecov Report
@@ Coverage Diff @@
## master #4904 +/- ##
=======================================
Coverage 76.65% 76.65%
=======================================
Files 158 158
Lines 10057 10057
Branches 1269 1269
=======================================
Hits 7709 7709
Misses 2007 2007
Partials 341 341
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This makes sense. Definitely something to test out more on our next QA pass, I'll add this to our list
Migration guide: https://github.com/stripe/stripe-python/wiki/Migration-guide-for-v2
https://github.com/stripe/stripe-python/blob/master/CHANGELOG.md
Related https://github.com/rtfd/readthedocs-corporate/pull/422